proto/frequenz/api/microgrid/v1alpha19/microgrid.proto:605

• The comment says “discharge power setpoint”, but the rest of this file consistently uses “set-point” (hyphenated). For consistency, consider changing this to “set-point”.

  // The preferred power source for the electrical component when setting a
  // discharge power setpoint. This is relevant for hybrid inverters which have
  // both PV and battery inputs added to them, and can use either of those

proto/frequenz/api/microgrid/v1alpha18/microgrid.proto:616

• power_source is a proto3 enum field, so callers can’t really “not provide” it in a distinguishable way (omitting it on the wire yields the default POWER_SOURCE_UNSPECIFIED). Consider updating the wording to reflect that POWER_SOURCE_UNSPECIFIED/omitted means PV-preferred, or make the field optional if presence needs to be detected.

  // If the above criteria are met, and this field is not provided, then the API
  // service will default to using `POWER_SOURCE_PV_PREFERRED` as the power
  // source.
  //

proto/frequenz/api/microgrid/v1alpha18/microgrid.proto:605

• The comment says “discharge power setpoint”, but the rest of this file consistently uses “set-point” (hyphenated). For consistency, consider changing this to “set-point”.

  // The preferred power source for the electrical component when setting a
  // discharge power setpoint. This is relevant for hybrid inverters which have
  // both PV and battery inputs added to them, and can use either of those

proto/frequenz/api/microgrid/v1alpha18/microgrid.proto:620

• In v1alpha18, power_source is field number 5 but it appears before fields 3 (power) and 4 (request_lifetime) in the message definition. Reordering the fields in the .proto to match numeric order would make the schema easier to read and less error-prone to maintain.

  PowerSource power_source = 5;

RELEASE_NOTES.md:18

• The release notes summary currently states that all new features are available exclusively in v1alpha19, but this PR also adds new API surface to the stable v1alpha18 package. Please update the earlier summary wording to avoid implying v1alpha18 is unchanged in this release.

## Stable `v1alpha18` Preview API

The `v1alpha18` API is also stable and has the following symbol additions:

proto/frequenz/api/microgrid/v1alpha19/microgrid.proto:616

• power_source is a proto3 enum field, so callers can’t really “not provide” it in a distinguishable way (omitting it on the wire yields the default POWER_SOURCE_UNSPECIFIED). Consider updating the wording to reflect that POWER_SOURCE_UNSPECIFIED/omitted means PV-preferred, or make the field optional if presence needs to be detected.

  // If the above criteria are met, and this field is not provided, then the API
  // service will default to using `POWER_SOURCE_PV_PREFERRED` as the power
  // source.
  //